Skip to content
This repository was archived by the owner on Jan 19, 2019. It is now read-only.

Add some default binding redirects to use on UWP #12

Merged
merged 1 commit into from
Oct 17, 2016
Merged

Add some default binding redirects to use on UWP #12

merged 1 commit into from
Oct 17, 2016

Conversation

bricelam
Copy link
Contributor

@bricelam bricelam commented Oct 17, 2016

@dnfclas
Copy link

dnfclas commented Oct 17, 2016

Hi @bricelam, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@@ -691,6 +691,8 @@ function GetCsprojArguments($startupProject, $outputFileName) {
$dataDirectory = Join-Path $startupProjectDir 'App_Data'
} elseif ($appConfig) {
$configurationFile = GetProperty $appConfig.Properties FullPath
} elseif (IsUwpProject $startupProject) {
$configurationFile = Join-Path $PSScriptRoot 'uap10.0/ef.exe.config'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note, these can be overridden by adding your own App.config to the startup project

@bricelam bricelam changed the base branch from dev to rel/1.1.0-preview1 October 17, 2016 20:20
@bricelam bricelam merged commit 0a26b3f into aspnet:rel/1.1.0-preview1 Oct 17, 2016
@bricelam bricelam deleted the redirect branch October 17, 2016 20:22
</dependentAssembly>
</assemblyBinding>
</runtime>
</configuration>
Copy link

@divega divega Oct 17, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am guessing this isn't enough to fix the other issues mentioned in dotnet/efcore#5171:

design-time migrations don't fail with a load error, but still fail due to getting the wrong versions of BCL libraries such as DataAnnotations. See dotnet/efcore#5945.
design-time tools are bound to 1.0.0 assembly versions but need to execute against other versions such as 1.0.1. On UWP, NuGet won't add the necessary binding redirects. See dotnet/efcore#6551.

Those seem to be show stoppers.

Copy link
Contributor Author

@bricelam bricelam Oct 18, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will enable 1.1 to work by default. 1.0 will still need to add overrides, but we may be able to do more there. We can't solve all the unification issues without a new architecture--too much of unification is done by NuGet at restore time now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants